Skip to content
This repository has been archived by the owner on Jul 22, 2024. It is now read-only.

mxnet recipe #134

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

mxnet recipe #134

wants to merge 1 commit into from

Conversation

knm3000
Copy link

@knm3000 knm3000 commented Oct 23, 2019

MXNet GPU python package (MXNet library and python bindings to it)

@jayfurmanek
Copy link
Contributor

@knm3000 Check this commit for a better way to avoid the 10min timeout.

3aaf374

@jayfurmanek
Copy link
Contributor

Nice! Looks like the last update allowed it complete! I'll review.

@knm3000
Copy link
Author

knm3000 commented Nov 26, 2019

@jayfurmanek It was the test commit using the new base image (for now from my personal docker hub https://hub.docker.com/repository/docker/knm3000/linux-anvil-cuda-ppc64le) to check 50 min limit.
This is not a final version. I put 3 cuda packages into that base image to enable the build within 50 min,
Now I need to fix the build for this PR conda-forge/docker-images#123

.travis.yml Show resolved Hide resolved
.travis.yml Show resolved Hide resolved
@@ -0,0 +1,16 @@
c_compiler:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then you can drop this file.

c_compiler_version:
- '7'
channel_sources:
- conda-forge,defaults
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note this order specifies priority. Does it still work if you put defaults first?

-e UPLOAD_PACKAGES \
-e CI \
-a stdin -a stdout -a stderr \
knm3000/linux-anvil-cuda-ppc64le \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you change this to $DOCKER_IMAGE? Then use this image name up in the .travis.yaml file.

@knm3000 knm3000 force-pushed the mxnet branch 2 times, most recently from a5a80a5 to 7914673 Compare December 5, 2019 10:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants